Skip to content

issue701 fix for l4l7 logical_interface unique ID #303

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

issue701 fix for l4l7 logical_interface unique ID #303

wants to merge 1 commit into from

Conversation

ogorczow
Copy link
Contributor

@ogorczow ogorczow commented Sep 2, 2025

There is an issue with generating ID for logical_interface. Fix is to improve ID by adding concrete_interface device into ID string:

---
apic:
  tenants:
    - name: 'l4l7'
      services:
        l4l7_devices:
          - name: PBR_FW
            service_type: FW
            physical_domain: phy_dom
            concrete_devices:
              - name: PBR_FW-A
                interfaces:
                  #id PBR_FW-A-PBR
                  - name: PBR
                    node_id: 101
                    node2_id: 102
                    channel: ipg-vpc-firewall_fw-a
              - name: PBR_FW-B
                interfaces:
                  #id PBR_FW-B-PBR
                  - name: PBR
                    node_id: 101
                    node2_id: 102
                    channel: ipg-vpc-firewall_fw-b
            logical_interfaces:
              - name: PBR
                vlan: 100
                concrete_interfaces:
                  #id PBR-PBR
                  - device: PBR_FW-A
                    interface_name: PBR
                  #id PBR-PBR <----------------------------- duplicated ID
                  - device: PBR_FW-B
                    interface_name: PBR

@ogorczow ogorczow requested a review from juchowan September 2, 2025 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@juchowan juchowan Awaiting requested review from juchowan

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ogorczow